Skip to content

fix(windows): hide console windows for AEC, text-monitor, and mic-listener#1234

Merged
gabrielste1n merged 3 commits into
OpenWhispr:mainfrom
stantheman0128:fix/win-aec-spawn-hide
Jul 19, 2026
Merged

fix(windows): hide console windows for AEC, text-monitor, and mic-listener#1234
gabrielste1n merged 3 commits into
OpenWhispr:mainfrom
stantheman0128:fix/win-aec-spawn-hide

Conversation

@stantheman0128

Copy link
Copy Markdown
Contributor

Summary

  • Several native Windows helpers were spawned without windowsHide: meeting AEC, text-edit monitor, and windows-mic-listener.exe.
  • Those flashes show up during meetings and dictation flows on Windows.
  • Pass windowsHide: true on those spawn sites (same pattern as whisper/ffmpeg/sidecar helpers).

AI assistance

This PR was prepared with AI assistance (Cursor/Grok). Human author: Stan Shih (@stantheman0128). Please review carefully.

Verification / Evidence

Focused tests:

node --test test/helpers/windows-helper-spawn-hide.test.js
# 3 pass, 0 fail

Test plan

  • Source regression for AEC, text-monitor, and Win32 mic-listener spawns
  • Maintainer: start a meeting / dictation edit path on Windows and confirm no helper console flash

…tener

Native Windows helper binaries were spawned without windowsHide, flashing
consoles during meeting AEC, text-edit monitoring, and mic activity detection.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gabrielste1n
gabrielste1n self-requested a review July 19, 2026 17:30

@gabrielste1n gabrielste1n left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and verified — the fix matches the established windowsHide pattern used across the other native helper spawns, and stdio behavior (including the mic-listener's stdin parent-death monitoring) is unchanged. I merged latest main into the branch, renamed the test to match the camelCase convention in test/helpers/, and tightened the assertions so windowsHide must appear inside each spawn's own options object. Full suite, lint, and typecheck are green. Thank you for the clean, well-scoped fix! 🙏

@gabrielste1n
gabrielste1n merged commit 5483bed into OpenWhispr:main Jul 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants